name: pre-commit run_id: commands[0] env HOME: /home/jenkins env LANG: C.UTF-8 env PATH: /w/workspace/bgpcep-tox-verify-master/.tox/pre-commit/bin:/opt/pyenv/bin:/tmp/venv-uQRs/bin:/opt/pyenv/shims:/home/jenkins/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/opt/puppetlabs/bin env PIP_DISABLE_PIP_VERSION_CHECK: 1 env PYTHONHASHSEED: 40888652 env PYTHONIOENCODING: utf-8 env SSH_AUTH_SOCK: ******************************** env TOX_ENV_DIR: /w/workspace/bgpcep-tox-verify-master/.tox/pre-commit env TOX_ENV_NAME: pre-commit env TOX_WORK_DIR: /w/workspace/bgpcep-tox-verify-master/.tox env VIRTUAL_ENV: /w/workspace/bgpcep-tox-verify-master/.tox/pre-commit env __TOX_ENVIRONMENT_VARIABLE_ORIGINAL_CI: true metadata pid: 2395 cwd: /w/workspace/bgpcep-tox-verify-master allow: /w/workspace/bgpcep-tox-verify-master/.tox/pre-commit/bin/* cmd: pre-commit run --all-files --show-diff-on-failure exit_code: 1 [WARNING] hook id `remove-tabs` uses deprecated stage names (commit) which will be removed in a future version. run: `pre-commit migrate-config` to automatically fix this. [INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks. [WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version. Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this. if it does not -- consider reporting an issue to that repo. [INFO] Initializing environment for https://github.com/Lucas-C/pre-commit-hooks. [INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/Lucas-C/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... trim trailing whitespace.................................................Failed - hook id: trailing-whitespace - exit code: 1 - files were modified by this hook Fixing tests/variables/bgpfunctional/l3vpn_ipv4/bgp-l3vpn-ipv4-manypeers.j2 Fixing tests/variables/bgpflowspec/bgp-flowspec-redirect-manypeers.j2 Fixing tests/variables/bgpfunctional/l3vpn_ipv4/bgp_l3vpn_ipv4_manypeers/data.json Fixing tests/variables/bgpflowspec/bgp_flowspec_redirect_manypeers/data.json Fixing tests/tools/exabgp_files/exarpc.py Tabs remover.............................................................Passed pre-commit hook(s) made changes. If you are seeing this message in CI, reproduce locally with: `pre-commit run --all-files`. To run `pre-commit` as part of git workflow, use `pre-commit install`. All changes made by hooks: diff --git a/tests/tools/exabgp_files/exarpc.py b/tests/tools/exabgp_files/exarpc.py index 71294f038f..fcc17eafa9 100755 --- a/tests/tools/exabgp_files/exarpc.py +++ b/tests/tools/exabgp_files/exarpc.py @@ -14,8 +14,8 @@ import threading from exabgp.bgp.message import Message from xmlrpc.server import SimpleXMLRPCServer from socketserver import ThreadingMixIn - - + + class ThreadedXMLRPCServer(ThreadingMixIn, SimpleXMLRPCServer): """Multi-threaded XMLRPC Server.""" pass @@ -396,7 +396,7 @@ def main(*argv): msg_queue = queue.Queue() t_worker = threading.Thread( - target=worker_loop, + target=worker_loop, args=(storage, msg_queue, in_args.encoder) ) t_worker.daemon = True diff --git a/tests/variables/bgpflowspec/bgp-flowspec-redirect-manypeers.j2 b/tests/variables/bgpflowspec/bgp-flowspec-redirect-manypeers.j2 index 2ad0f77938..e06cb417cf 100644 --- a/tests/variables/bgpflowspec/bgp-flowspec-redirect-manypeers.j2 +++ b/tests/variables/bgpflowspec/bgp-flowspec-redirect-manypeers.j2 @@ -2,7 +2,7 @@ neighbor {{ ODL_IP }} { router-id 127.0.1.{{ i }}; local-address 127.0.1.{{ i }}; - + local-as 64496; peer-as 64496; hold-time 180; diff --git a/tests/variables/bgpflowspec/bgp_flowspec_redirect_manypeers/data.json b/tests/variables/bgpflowspec/bgp_flowspec_redirect_manypeers/data.json index 4400c73aab..70efb4b9cc 100644 --- a/tests/variables/bgpflowspec/bgp_flowspec_redirect_manypeers/data.json +++ b/tests/variables/bgpflowspec/bgp_flowspec_redirect_manypeers/data.json @@ -623,4 +623,3 @@ ] } } - \ No newline at end of file diff --git a/tests/variables/bgpfunctional/l3vpn_ipv4/bgp-l3vpn-ipv4-manypeers.j2 b/tests/variables/bgpfunctional/l3vpn_ipv4/bgp-l3vpn-ipv4-manypeers.j2 index c54ae90b13..e06abc4fad 100755 --- a/tests/variables/bgpfunctional/l3vpn_ipv4/bgp-l3vpn-ipv4-manypeers.j2 +++ b/tests/variables/bgpfunctional/l3vpn_ipv4/bgp-l3vpn-ipv4-manypeers.j2 @@ -2,7 +2,7 @@ neighbor {{ ODL_IP }} { router-id 127.0.1.{{ i }}; local-address 127.0.1.{{ i }}; - + local-as 64496; peer-as 64496; hold-time 180; diff --git a/tests/variables/bgpfunctional/l3vpn_ipv4/bgp_l3vpn_ipv4_manypeers/data.json b/tests/variables/bgpfunctional/l3vpn_ipv4/bgp_l3vpn_ipv4_manypeers/data.json index 3d86775d75..e4536486b4 100644 --- a/tests/variables/bgpfunctional/l3vpn_ipv4/bgp_l3vpn_ipv4_manypeers/data.json +++ b/tests/variables/bgpfunctional/l3vpn_ipv4/bgp_l3vpn_ipv4_manypeers/data.json @@ -444,4 +444,3 @@ ] } } - \ No newline at end of file